Skip to content

Comments

[2201.12.x] Add new scan rules#614

Merged
TharmiganK merged 2 commits into2201.12.xfrom
static-rules-12.x
Oct 22, 2025
Merged

[2201.12.x] Add new scan rules#614
TharmiganK merged 2 commits into2201.12.xfrom
static-rules-12.x

Conversation

@TharmiganK
Copy link
Contributor

@TharmiganK TharmiganK commented Oct 22, 2025

Purpose

$Subject

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility

@codecov
Copy link

codecov bot commented Oct 22, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 79.87616% with 65 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.83%. Comparing base (f299f7d) to head (cef79b6).
⚠️ Report is 3 commits behind head on 2201.12.x.

Files with missing lines Patch % Lines
...mpiler/staticcodeanalyzer/CryptoAnalyzerUtils.java 72.64% 7 Missing and 25 partials ⚠️
...o/compiler/staticcodeanalyzer/FunctionContext.java 79.31% 6 Missing and 6 partials ⚠️
...ctionrules/AvoidReusingCounterModeVectorsRule.java 77.14% 2 Missing and 6 partials ⚠️
...zer/functionrules/AvoidFastHashAlgorithmsRule.java 80.00% 2 Missing and 5 partials ⚠️
...ticcodeanalyzer/CryptoCipherAlgorithmAnalyzer.java 88.23% 1 Missing and 1 partial ⚠️
...a/stdlib/crypto/compiler/CryptoCompilerPlugin.java 80.00% 0 Missing and 1 partial ⚠️
.../staticcodeanalyzer/CryptoFunctionRulesEngine.java 93.75% 0 Missing and 1 partial ⚠️
...b/crypto/compiler/staticcodeanalyzer/RuleImpl.java 88.88% 1 Missing ⚠️
...r/functionrules/AvoidWeakCipherAlgorithmsRule.java 92.85% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@               Coverage Diff               @@
##             2201.12.x     #614      +/-   ##
===============================================
- Coverage        83.57%   82.83%   -0.75%     
- Complexity         228      347     +119     
===============================================
  Files               30       42      +12     
  Lines             1291     1614     +323     
  Branches           181      253      +72     
===============================================
+ Hits              1079     1337     +258     
- Misses             168      187      +19     
- Partials            44       90      +46

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

ThisaruGuruge
ThisaruGuruge reviewed Oct 22, 2025
View reviewed changes
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 22, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@TharmiganK TharmiganK merged commit 480c033 into 2201.12.x Oct 22, 2025
5 of 6 checks passed
@TharmiganK TharmiganK deleted the static-rules-12.x branch October 22, 2025 08:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ThisaruGuruge ThisaruGuruge ThisaruGuruge left review comments

@daneshk daneshk daneshk approved these changes

@MohamedSabthar MohamedSabthar Awaiting requested review from MohamedSabthar MohamedSabthar is a code owner

@DimuthuMadushan DimuthuMadushan Awaiting requested review from DimuthuMadushan DimuthuMadushan is a code owner

@shafreenAnfar shafreenAnfar Awaiting requested review from shafreenAnfar shafreenAnfar is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@TharmiganK @daneshk @ThisaruGuruge @nureka-rodrigo